Skip to content

Pass options as keywords during block extraction#521

Merged
lessthanjacob merged 4 commits intoprocore-oss:mainfrom
tylerhunt:keyword-field-options
Feb 25, 2026
Merged

Pass options as keywords during block extraction#521
lessthanjacob merged 4 commits intoprocore-oss:mainfrom
tylerhunt:keyword-field-options

Conversation

@tylerhunt
Copy link
Contributor

Allows field definitions to use keyword options for blocks:

  field :vehicle_make do |_obj, vehicle:, **|
    "#{vehicle[:make]}"
  end

@tylerhunt tylerhunt requested review from a team and ritikesh as code owners May 30, 2025 19:43
@jhollinger jhollinger force-pushed the keyword-field-options branch from 1c21c38 to cf57240 Compare July 8, 2025 20:05
lessthanjacob
lessthanjacob previously approved these changes Jul 8, 2025
jhollinger
jhollinger previously approved these changes Jul 22, 2025
@tylerhunt tylerhunt force-pushed the keyword-field-options branch from cf57240 to fb7fb0c Compare July 23, 2025 01:15
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Signed-off-by: Tyler Hunt <tyler@tylerhunt.com>
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@tylerhunt tylerhunt dismissed stale reviews from jhollinger and lessthanjacob via 2f9b820 February 21, 2026 19:22
@tylerhunt
Copy link
Contributor Author

Merged upstream. Looks like that dismissed the reviews. Any blocker to getting this merged?

@lessthanjacob
Copy link
Contributor

@tylerhunt Apologies for the delay here!

Once the checks pass, I'll go ahead and merge this.

@lessthanjacob
Copy link
Contributor

@tylerhunt not quite sure why it's picking up rubocop violations on this branch, but I went ahead and fixed it over here: #570. I'll let you know when that's merged in.

@lessthanjacob lessthanjacob merged commit ac39daa into procore-oss:main Feb 25, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants